-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding in support for custom configurations #18
base: master
Are you sure you want to change the base?
Conversation
@e-liner - This looks good to me and a (hopefully) useful contribution to the upstream buildroot. The master branch in this repository tracks upstream. I think we need this pull request to be against 2016.02-op-build instead. We have tried to keep the op-build pretty close to master and then contribute the changes to the upstream buildroot. 2016.02-op-build is the most recent branch @shenki is maintaining for op-build. @shenki - Can you review and give @e-liner advice on how to handle this? |
Hey @e-liner, thanks for the patch - I think this is a great idea. I tend to merge patches in myself rather than using the github PR button, don't worry about re-doing the pull request against our -op-build branch. If you have time, please submit it to upstream: https://buildroot.org/downloads/manual/manual.html#submitting-patches If you're busy then I can submit it on your behalf. The buildroot maintainers very responsive, so we will wait for their review, and then I will pick up the patch from upstream. |
@shenki - Submitted to upstream here: http://lists.busybox.net/pipermail/buildroot/2016-April/158666.html . I'll let you know if I see it merged in the upstream master. |
It looks like upstream rejected this? |
Yes, upstream decided that this was too specific because it included the custom directory in the Makefile, they decided it would be better to add in a wildcard and then on our op-build side we can have a symlink from openpower/configs/custom to openpower/custom/configs. |
This change will add in the custom _defconfig's to the buildroot makefile so that when a _defconfig is in the custom directory it is recognized correctly, and is able to be used.
Changelog (since 883b726): - 3e020f7 Add 43436S firmware Future Zero 2 W boards will have an alternative version of 43436 that should have different firmware, named brcmfmac43436s-sdio. - 3888ba2 Merge pull request open-power#18 from HiassofT/43436S Signed-off-by: Peter Seiderer <[email protected]> Signed-off-by: Yann E. MORIN <[email protected]>
Changelog (since 883b726): - 3e020f7 Add 43436S firmware Future Zero 2 W boards will have an alternative version of 43436 that should have different firmware, named brcmfmac43436s-sdio. - 3888ba2 Merge pull request open-power#18 from HiassofT/43436S Signed-off-by: Peter Seiderer <[email protected]> Signed-off-by: Yann E. MORIN <[email protected]>
This change will add in the custom _defconfig's to the buildroot
makefile so that when a _defconfig is in the custom directory it is
recognized correctly, and is able to be used.
This change is